-
Notifications
You must be signed in to change notification settings - Fork 4.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
CAMEL-20019: add platform http vertx cookie handler #13301
CAMEL-20019: add platform http vertx cookie handler #13301
Conversation
🌟 Thank you for your contribution to the Apache Camel project! 🌟 🤖 CI automation will test this PR automatically. 🐫 Apache Camel Committers, please review the following items:
|
@davsclaus, were you thinking of something along these lines for the Vertx cookie hander? |
Yes that is really good. |
Description
A simple cookie handler that allows cookie addition, retrieval and expiry/removal.
Using the cookieJar that is available via the Vertx routing-context seemed to the be simplest implementation, and add/remove/get cookie operations delegate to the request/response Vertx objects.
Get/Remove operations on the CookieHandler return String values as opposed to Cookies to abstract away from specific Cookie implementations
The cookie handler is made available to routes as the 'COOKIE_HANDLER' exchange property.
Target
camel-3.x
, whereas Camel 4 uses themain
branch)Tracking
Apache Camel coding standards and style
mvn clean install -DskipTests
locally and I have committed all auto-generated changes